[2/4] fbmem: Check virtual screen sizes in fb_set_var()
authorHelge Deller <deller@gmx.de>
Wed, 29 Jun 2022 13:53:55 +0000 (15:53 +0200)
committerBen Hutchings <benh@debian.org>
Sat, 23 Jul 2022 22:32:10 +0000 (23:32 +0100)
Origin: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit?id=b81212828ad19ab3eccf00626cd04099215060bf
Bug-Debian-Security: https://security-tracker.debian.org/tracker/CVE-2021-33655

commit 6c11df58fd1ac0aefcb3b227f72769272b939e56 upstream.

Verify that the fbdev or drm driver correctly adjusted the virtual
screen sizes. On failure report the failing driver and reject the screen
size change.

Signed-off-by: Helge Deller <deller@gmx.de>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: stable@vger.kernel.org # v5.4+
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Gbp-Pq: Topic bugfix/all
Gbp-Pq: Name fbmem-check-virtual-screen-sizes-in-fb_set_var.patch

drivers/video/fbdev/core/fbmem.c

index bf61770f6f6b987534a60e2c54a0c1e9d1126ee3..213e738d9eb77edaa90316a4655f0d860e6406f0 100644 (file)
@@ -1019,6 +1019,16 @@ fb_set_var(struct fb_info *info, struct fb_var_screeninfo *var)
        if (ret)
                return ret;
 
+       /* verify that virtual resolution >= physical resolution */
+       if (var->xres_virtual < var->xres ||
+           var->yres_virtual < var->yres) {
+               pr_warn("WARNING: fbcon: Driver '%s' missed to adjust virtual screen size (%ux%u vs. %ux%u)\n",
+                       info->fix.id,
+                       var->xres_virtual, var->yres_virtual,
+                       var->xres, var->yres);
+               return -EINVAL;
+       }
+
        if ((var->activate & FB_ACTIVATE_MASK) != FB_ACTIVATE_NOW)
                return 0;